Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: Add test for GitHub Actions dependabot configuration #157

Merged
merged 1 commit into from
Jun 1, 2024

Conversation

zkoppert
Copy link
Member

@zkoppert zkoppert commented May 31, 2024

Pull Request

Proposed Changes

Fixes #156

This pull request introduces tests for the GitHub Actions dependabot configuration in the test_dependabot_file.py file, enhancing the test coverage for dependabot configurations.

  • Adds new tests: Implements two new test methods, test_build_dependabot_file_with_github_actions and test_build_dependabot_file_with_github_actions_without_files.
    • The first test ensures that the dependabot configuration for GitHub Actions is correctly generated when YAML files are present in the .github/workflows directory.
    • The second test verifies that the dependabot configuration is None when no YAML files are found in the .github/workflows directory, aligning with the expected behavior for repositories without GitHub Actions workflows.

Readiness Checklist

Author/Contributor

  • If documentation is needed for this change, has that been included in this pull request
  • run make lint and fix any issues that you have introduced
  • run make test and ensure you have test coverage for the lines you are introducing
  • If publishing new data to the public (scorecards, security scan results, code quality results, live dashboards, etc.), please request review from @jeffrey-luszcz

Reviewer

  • Label as either fix, documentation, enhancement, infrastructure, maintenance or breaking

@zkoppert zkoppert merged commit 32dd499 into main Jun 1, 2024
28 checks passed
@zkoppert zkoppert deleted the github-actions-tests branch June 1, 2024 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add github actions test
2 participants